Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for bug #18556 #79

Closed
wants to merge 1 commit into from
Closed

Fix for bug #18556 #79

wants to merge 1 commit into from

Conversation

netnet
Copy link

@netnet netnet commented May 10, 2012

Patch to avoid locale-aware lowercasing for letter 'I' in zend_str_tolower() and friends.

@avar
Copy link

avar commented Jul 2, 2012

This fix looks like a crazy hack, the real solution to this is to change the comparison functions for identifiers not to be locale dependent, not hack around a specific locale and its treatment of the letter "i" in a low-level copying function.

@php-pulls
Copy link

Comment on behalf of stas at php.net:

other fix will be done.

@php-pulls php-pulls closed this Jul 14, 2012
php-pulls pushed a commit that referenced this pull request Apr 1, 2013
php-pulls pushed a commit that referenced this pull request Apr 1, 2013
* PHP-5.5:
  Fixed issue #79 (Optimization Problem/Bug)
  fix coverage data for the opcache optimizer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants